system: Don't leave overview when activating most items

The current behavior wasn't designed, but was introduced in commit
84efaac52b to work around technical limitations when we were still
using external tools like gnome-screensaver or gnome-session-quit
to handle the actions. Those limitations are long gone, so it makes
sense to make the actions consistent with the corresponding keyboard
shortcuts: Leave the overview when launching an application, and leave
it alone otherwise.

https://bugzilla.gnome.org/show_bug.cgi?id=691900

diff --git a/js/ui/status/system.js b/js/ui/status/system.js
index 604183f69f8dd41eb9cc756e33db3397f42d84e0..27c9e238733503970588026fa5e6989e1c9de073 100644 (file)
--- a/js/ui/status/system.js
+++ b/js/ui/status/system.js
@@ -445,13 +445,11 @@ var Indicator = new Lang.Class({
 
     _onLockScreenClicked: function() {
         this.menu.itemActivated(BoxPointer.PopupAnimation.NONE);
-        Main.overview.hide();
         Main.screenShield.lock(true);
     },
 
     _onLoginScreenActivate: function() {
         this.menu.itemActivated(BoxPointer.PopupAnimation.NONE);
-        Main.overview.hide();
         if (Main.screenShield)
             Main.screenShield.lock(false);
 
@@ -462,13 +460,11 @@ var Indicator = new Lang.Class({
     },
 
     _onQuitSessionActivate: function() {
-        Main.overview.hide();
         this._session.LogoutRemote(0);
     },
 
     _onPowerOffClicked: function() {
         this.menu.itemActivated();
-        Main.overview.hide();
         this._session.ShutdownRemote(0);
     },